Skip to content

ci: added concurrently to build and deploy jobs#501

Merged
agliga merged 4 commits intomainfrom
concurrent
Feb 12, 2026
Merged

ci: added concurrently to build and deploy jobs#501
agliga merged 4 commits intomainfrom
concurrent

Conversation

@agliga
Copy link
Collaborator

@agliga agliga commented Feb 6, 2026 •
edited
Loading

Build was taking too long
So I made our builds run in pararallel! I used concurrently here

  • Updated build to run skin first, and then mtc (for building the evo-site), ebayui-core, ebayui-core-react in paraellel. Also updated ebayui-core react to run smoke tests in parallel
  • Updated deploy to run in parallel. Each site will be built at the same time
    Its a bit more verbose but I think its good
  • Updated start in the root to run concurrently so all servers are started (on different ports)
  • Removed some unused commands
command old time new time
npm run build 1:51 1:16
npm run deploy 2:06 1:01

Bonus change

  • removed some console logs
  • ebayui-core-react Changed npm test to do vitest and npm run build to do vitest run (so no hanging)

Copilot AI review requested due to automatic review settings February 6, 2026 19:34
@changeset-bot
Copy link

changeset-bot bot commented Feb 6, 2026 •
edited
Loading

⚠️ No Changeset found

Latest commit: 37f722a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copilot AI reviewed Feb 6, 2026
View reviewed changes
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Speed up CI/local workflows by parallelizing build, deploy, and start commands using concurrently, and remove a few debug logs.

Changes:

  • Parallelized root build, deploy, and start scripts via concurrently.
  • Updated ebayui-core and ebayui-core-react scripts to run more tasks in parallel / avoid hanging tests.
  • Removed debug <log>/console.log statements.

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/tags/a11y-docs-list.marko Removes debug logging from the a11y docs list rendering.
src/data/components.ts Removes console logging from URL metadata parsing.
packages/ebayui-core/package.json Parallelizes build steps with concurrently to reduce build time.
packages/ebayui-core-react/package.json Adjusts test/build scripts and runs smoke tests in parallel.
package.json Adds concurrently and parallelizes root build/deploy/start workflows.

@agliga
Copy link
Collaborator Author

agliga commented Feb 9, 2026

Faulty tests should be fixed by this PR
#503

agliga and others added 3 commits February 9, 2026 14:43
@agliga agliga self-assigned this Feb 12, 2026
@agliga agliga merged commit d8bd3c0 into main Feb 12, 2026
3 checks passed
@agliga agliga deleted the concurrent branch February 12, 2026 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@LuLaValva LuLaValva LuLaValva approved these changes

@ArtBlue ArtBlue Awaiting requested review from ArtBlue

@HenriqueLimas HenriqueLimas Awaiting requested review from HenriqueLimas

Assignees

@agliga agliga

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@agliga @LuLaValva